Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Added option to skip stripping elf #20

Merged
merged 2 commits into from
Apr 2, 2024
Merged

feat: Added option to skip stripping elf #20

merged 2 commits into from
Apr 2, 2024

Conversation

nikarh
Copy link
Member

@nikarh nikarh commented Apr 2, 2024

Even though in most cases stripping should be enabled since a hello world binary with std with debug symbols yields a ~13mb elf for Vita, debugging isolated problems could benefit from dwarf symbols.

This PR adds a flag readable from Cargo.toml which can toggle stripping on and off (with on by default)

@nikarh nikarh merged commit 8af5c33 into main Apr 2, 2024
6 checks passed
@nikarh nikarh deleted the skip-strip branch April 2, 2024 18:30
@vita-rust-bot vita-rust-bot bot mentioned this pull request Apr 2, 2024
nikarh added a commit that referenced this pull request Apr 2, 2024
# Changelog

## [0.2.1] - 2024-04-02
### Details
#### Changed
- Bump peter-evans/create-pull-request from 5 to 6 (#15) by
@dependabot[bot] in #15
- Bump softprops/action-gh-release from 1 to 2 (#16) by @dependabot[bot]
in #16
- Bump mathieudutour/github-tag-action from 6.1 to 6.2 (#17) by
@dependabot[bot] in #17
- Added cargo-deny, enabled pedantic clippy, upgraded dependencies (#18)
by @nikarh in #18
- Added option to skip stripping elf (#20) by @nikarh in #20

#### Fixed
- Fixed publish pipeline permissions by @nikarh
- Better error message when vita-parse-core is not found (#19) by
@nikarh in #19

Co-authored-by: nikarh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant